add plotPCA test - #1374
Conversation
|
I have a general question, would it be good to have those tests added to the current deeptools version as well. So that we can compare results? |
|
what do you mean by the current vesion? 3.x? |
Yea, that's the idea, though only for those functionalities that changed (bamCompare / bamCoverage, computeMatrix & multiBamSummary). Note that this doesn't need to be incoporated in the current version, as for now all the 'old' functionality is still present in 4.0.0 (either through the API, or by bringing back the entrypoints..). Note that the planemo tests at this point serve as a first checkpoint in this regard, as soon as rewrite is finished (and planemo tests hopefully pass), we can move to the more elaborate setup. |
Both existing tests only compared the rendered PNG. Adds numeric assertions on the tabular outputs, filling gaps found while triaging the stalled #1372/#1374 tests: - plotPCA: verify --outFileNameData (the existing test passed this flag but the check was commented out). Asserts the sign-independent eigenvalue column and table shape. - plotFingerprint: verify --outQualityMetrics + --JSDsample (AUC and JS Distance), neither previously tested anywhere. Expected values derived by running the current tools; verified with `pixi run pytest`. Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
4 participants
@WardDeb
.npzis in.gitignore. Neverthless, I added a required.npztest input file.